Skip to content

Added HostPath, ImagePullSecret and TerminationGracePeriodSeconds support #660

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Conversation

ofirmakmal
Copy link
Contributor

@ofirmakmal ofirmakmal commented Jun 20, 2023

Added additional fields needed for the Helm chart.

@vincentsarago
Copy link
Member

@ividito @ranchodeluxe would you be able to review this PR? 🙏

@ofirmakmal ofirmakmal changed the title Added hostpath, imagepullsecret and termination grace priod support. Added HostPath, ImagePullSecret and TerminationGracePeriodSeconds support Jun 20, 2023
Copy link
Contributor

@geospatial-jeff geospatial-jeff left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@ranchodeluxe ranchodeluxe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Left a tiny comment that can be ignored or updated

# mountPropagation: HostToContainer # OPTIONAL

imagePullSecrets: []

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nitpick: but I like what you did above with extraHostPathMounts showing an example of what should go there. Can we add # -name: regcred here also please as an example?

@ranchodeluxe ranchodeluxe merged commit d405395 into developmentseed:main Jun 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants